Remove "3 pubkeys in top level multisig" limit#1151
Remove "3 pubkeys in top level multisig" limit#1151dennisreimann wants to merge 1 commit intoMetacoSA:masterfrom dennisreimann:patch-1
Conversation
|
This limit is taken from original implementation in bitcoind Why you want to use bare multisig? IIUC it is deprecated and you should use |
|
This isn't a bare multisig, I'm coming across it in a But now that you hinted at that, I think the error isn't in the implementation here, but in our |
|
@joemphilips I was able to fix this issue on our end — thanks for the pointer, you are correct! |
* Fix Output Descriptor parsing for WSH multisig case Reuse existing function for extracting from a multisig descriptor, instead of recursively parsing the inner output descriptor. The latter would run into invalid cases, because it'd be interpreted as bare multisig, which supports only up to three public keys. For further details see MetacoSA/NBitcoin#1151. * Add CanParseDerivationSchemes test
2 participants
This was introduced in #696 and it seems like an arbitrary limit.
@joemphilips What is the reason for this?